Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactoring #558

Merged
merged 1 commit into from
Dec 17, 2021
Merged

Refactoring #558

merged 1 commit into from
Dec 17, 2021

Conversation

nervo
Copy link
Member

@nervo nervo commented Aug 16, 2021

- apt:
    name: [foo, bar]
-> ok: [debian.bullseye]

- apt:
    name: "{{ item }}"
  loop:
    - [foo, bar]
-> ok: [debian.bullseye] => (item=['foo', 'bar'])

@nervo nervo added the wip label Aug 16, 2021
@nervo nervo force-pushed the refactoring branch 14 times, most recently from fb15e8c to 8cb3d9f Compare August 16, 2021 14:51
@codecov-commenter
Copy link

codecov-commenter commented Aug 16, 2021

Codecov Report

❗ No coverage uploaded for pull request base (master@1f706fa). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #558   +/-   ##
=========================================
  Coverage          ?   94.64%           
=========================================
  Files             ?       97           
  Lines             ?     3924           
  Branches          ?      612           
=========================================
  Hits              ?     3714           
  Misses            ?      156           
  Partials          ?       54           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1f706fa...a490637. Read the comment docs.

@nervo nervo force-pushed the refactoring branch 2 times, most recently from 7b5d001 to 7d19276 Compare August 16, 2021 15:45
@nervo nervo marked this pull request as draft August 16, 2021 15:57
@nervo nervo force-pushed the refactoring branch 11 times, most recently from 50b508f to a4ac3d8 Compare August 18, 2021 13:55
Fix docker containers staten

Update ohmyzsh users template

fix cron empty lines

handle proftpd config parameter none value

more stable files url tests

Meta (#578)

* Change required ansible version in meta

* add collection meta folder and move runtime.yml

Release process (#579)

* Change required ansible version in meta

* add collection meta folder and move runtime.yml

* Release and publish process

Readme(s) (#576)

* Role variables

* Licence

* Other role references

* generic collection usage

* Set Ansible 2.9 as minimum target

* harmonize general readme

* link to main readme for installation instructions

* remove handlerz

* Remove handlers in readme

Co-authored-by: Yves Heitz <[email protected]>
@nervo nervo removed the wip label Dec 17, 2021
@nervo nervo marked this pull request as ready for review December 17, 2021 14:05
@gfaivre gfaivre merged commit fa19411 into master Dec 17, 2021
@gfaivre gfaivre deleted the refactoring branch December 17, 2021 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants